Skip to content

refactor: migrate make:command to AbstractGeneratorCommand - #10543

Open
paulbalandan wants to merge 1 commit into
codeigniter4:4.8from
paulbalandan:command-generator
Open

paulbalandan wants to merge 1 commit into
codeigniter4:4.8from
paulbalandan:command-generator

Conversation

@paulbalandan

Copy link
Copy Markdown
Member

Description
Follow up to #10524 .

Behavior change: generated commands are now modern, both for basic and generator commands. I added a way to re-point to the old commands for those who want. But I don't think we need that. Let me know what you think.

Checklist:

  • Securely signed commits
  • Component(s) with PHPDoc blocks, only if necessary or adds value (without duplication)
  • Unit testing, with >80% coverage
  • User guide updated
  • Conforms to style guide

@paulbalandan paulbalandan added the refactor Pull requests that refactor code label Sep 12, 2026
@github-actions github-actions Bot added the 4.8 PRs that target the `4.8` branch. label Sep 12, 2026
@paulbalandan
paulbalandan force-pushed the command-generator branch 2 times, most recently from 1c01884 to 8903b1d Compare September 12, 2026 17:18

@michalsn michalsn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should only support executing old commands. Generating them isn't something we should care about.

@carson-codeigniter4 carson-codeigniter4 Bot added the stale Pull requests with conflicts label Sep 15, 2026
@carson-codeigniter4

This comment has been minimized.

@carson-codeigniter4 carson-codeigniter4 Bot removed the stale Pull requests with conflicts label Sep 15, 2026
@paulbalandan

Copy link
Copy Markdown
Member Author

I think we should only support executing old commands. Generating them isn't something we should care about.

Updated. Removed the wording for support for generating legacy commands.

@carson-codeigniter4 carson-codeigniter4 Bot added the stale Pull requests with conflicts label Sep 15, 2026
@carson-codeigniter4 carson-codeigniter4 Bot removed the stale Pull requests with conflicts label Sep 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4.8 PRs that target the `4.8` branch. refactor Pull requests that refactor code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants